Skip to content

Conversation

@hugovk
Copy link
Member

@hugovk hugovk commented Feb 2, 2024

  • Add Dependabot to keep GitHub Actions up-to-date
  • Update pre-commit, add more handy checks, remove some that don't apply
  • Move isort config to pyproject.toml and specify Black profile for compatibility
  • Add link to PR of RTD docs preview

Comment on lines +7 to +10
groups:
actions:
patterns:
- "*"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Any reason for the grouping? Sometimes deps are related and must be updated together, but other times one update is problematic and it will block the others.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Mainly to reduce the total number of PRs to reduce "alert fatigue". I and many others get lots of PR/issue notifications, and these multiply on Dependabot days.

Other projects I maintain use grouped updates, and very occasionally one update may be problematic, but like you say, it goes both ways and sometimes they need grouping.

The vast majority of the time, grouped updates are fine.

Multiple PRs use up more CI time, and are also more cumbersome: after merging one, often it needs to rebase all the others, using more CI time, and needing to wait before merge.

Co-authored-by: Ezio Melotti <ezio.melotti@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants